Skip to content

Use TableLayoutPanel and FlowLayoutPanel for InsertEntryDlg layout#781

Merged
jasonleenaylor merged 1 commit intomainfrom
claude/infallible-tesla
Mar 24, 2026
Merged

Use TableLayoutPanel and FlowLayoutPanel for InsertEntryDlg layout#781
jasonleenaylor merged 1 commit intomainfrom
claude/infallible-tesla

Conversation

@jasonleenaylor
Copy link
Contributor

@jasonleenaylor jasonleenaylor commented Mar 23, 2026

  • Use TableLayoutPanel for overall dialog layout and FlowLayoutPanel for button row
  • Add FlowLayoutPanel inside MSAGroupBox so Grammatical Info controls flow left-to-right and collapse gracefully
    when fewer panels are visible
  • Add FlowLayoutPanel inside Gloss GroupBox to prevent overlap between the gloss text box and the Inflectional Affix
    Gloss Builder link
  • Fix ReplaceTextBoxWithMultiStringBox to preserve control order in FlowLayoutPanels

This change is Reviewable

* Enable AutoSize on m_propsGroupBox so combo boxes are not cut off when
  their font is set to 12pt at runtime.
* Dock the internal m_groupBox in MSAGroupBox to Fill so it stretches
  horizontally with the dialog.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link

NUnit Tests

    1 files  ±0      1 suites  ±0   5m 55s ⏱️ +7s
4 074 tests ±0  4 003 ✅ ±0  71 💤 ±0  0 ❌ ±0 
4 083 runs  ±0  4 012 ✅ ±0  71 💤 ±0  0 ❌ ±0 

Results for commit 2956711. ± Comparison against base commit e58af68.

@jasonleenaylor
Copy link
Contributor Author

Lexicon Edit:
image
Texts & Words Analyze:
image
Create new inflectional affix:
image

Copy link
Contributor

@mark-sil mark-sil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mark-sil reviewed 3 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on jasonleenaylor).

@jasonleenaylor jasonleenaylor merged commit bac16b8 into main Mar 24, 2026
7 checks passed
@jasonleenaylor jasonleenaylor deleted the claude/infallible-tesla branch March 24, 2026 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants